Skip to content

Conversation

Jack251970
Copy link

@Jack251970 Jack251970 commented Feb 27, 2025

Hello, firstly apologies for my misunderstanding of #1001. The solution should not be such simple.

After looking into related documents (https://stackoverflow.com/questions/74472659/exception-info-system-platformnotsupportedexception-eventlog-access-is-not-sup), I think this issue is related to the version of System.Diagnostics.EventLog package. We should specific EventLog package version for each .Net versions.

I have already tested in Flow-Launcher/Flow.Launcher#3283 and now TaskScheduler works fine with our project based on .net7 and we are looking forward to your new version!

@dahall
Copy link
Owner

dahall commented Mar 1, 2025

Thanks for the work on this. I just pushed an update with your ideas, just implemented in a Choose/When statement so it is easier for me to see.

@dahall dahall closed this Mar 1, 2025
@Jack251970
Copy link
Author

@dahall Thanks for your update. But it seems that 1.12.1 is unavailable on nuget.

@dahall
Copy link
Owner

dahall commented Mar 3, 2025

They are not published there yet. They are available on AppVeyor. See homepage for details on pulling them. I'd appreciate your review before I publish to NuGet.org.

@Jack251970
Copy link
Author

They are not published there yet. They are available on AppVeyor. See homepage for details on pulling them. I'd appreciate your review before I publish to NuGet.org.

Get it. I will wait for this release.

For now, it works fine for our .Net7 project!

@dahall
Copy link
Owner

dahall commented Mar 4, 2025

So you have tested the 1.12.1 package and it works for you? If so, I'll push to NuGet.org.

@Jack251970
Copy link
Author

So you have tested the 1.12.1 package and it works for you? If so, I'll push to NuGet.org.

I’ve tested your latest changes, and everything works as expected. 👍

@dahall
Copy link
Owner

dahall commented Mar 5, 2025

Thank you!! I'll release to NuGet shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants